Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes incorrectly missing annotations #561

Merged
merged 2 commits into from
Nov 23, 2021

Conversation

lpetre
Copy link
Contributor

@lpetre lpetre commented Nov 23, 2021

Summary

In #546 I incorrectly stop marking some type annotations

Test Plan

python -m unittest libcst/metadata/tests/test_scope_provider.py

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 23, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #561 (300de78) into main (3895925) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #561      +/-   ##
==========================================
- Coverage   94.77%   94.77%   -0.01%     
==========================================
  Files         235      235              
  Lines       23274    23273       -1     
==========================================
- Hits        22058    22057       -1     
  Misses       1216     1216              
Impacted Files Coverage Δ
libcst/metadata/scope_provider.py 95.44% <ø> (-0.02%) ⬇️
libcst/metadata/tests/test_scope_provider.py 99.88% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3895925...300de78. Read the comment docs.

@zsol zsol merged commit 58b447d into Instagram:main Nov 23, 2021
@lpetre lpetre deleted the fix_missing_annotations branch January 7, 2022 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants